-
Notifications
You must be signed in to change notification settings - Fork 75
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve polygon and line documentation. #824
Conversation
This should make it clearer what the position, line, and polygon accessors do. It also explicitly defines a polygon data type.
src/lineFeature.js
Outdated
* @param {object|function} [val] If specified, use this for the line | ||
* accessor and return the feature. If not specified, return the current | ||
* line. If a function, this is passed `(dataElement, dataIndex)` and | ||
* returns an array of vertex elements. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
returns or return? above the sentences read return.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a possibility that this sentence could be improved further (and the ones below)?
521d317
to
d715877
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for consideration.
This should make it clearer what the position, line, and polygon accessors do. It also explicitly defines a polygon data type.